Skip to content

Add chart diffing GHA workflow#3281

Draft
samsimpson1 wants to merge 3 commits into
mainfrom
samsimpson1/chart-diff
Draft

Add chart diffing GHA workflow#3281
samsimpson1 wants to merge 3 commits into
mainfrom
samsimpson1/chart-diff

Conversation

@samsimpson1
Copy link
Copy Markdown
Member

No description provided.

Comment on lines +11 to +26
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
ref: ${{ github.event.pull_request.base.ref }}
show-progress: false
- name: Run govuk-app-render
run: ./govuk-app-render.sh
- name: Archive rendered charts
uses: actions/upload-artifact@v4
with:
name: rendered-charts-base
path: output/
retention-days: 1
render-charts-head:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
Comment on lines +27 to +42
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
ref: ${{ github.event.pull_request.head.ref }}
show-progress: false
- name: Run govuk-app-render
run: ./govuk-app-render.sh
- name: Archive rendered charts
uses: actions/upload-artifact@v4
with:
name: rendered-charts-head
path: output/
retention-days: 1
diff-charts:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
@samsimpson1 samsimpson1 force-pushed the samsimpson1/chart-diff branch 4 times, most recently from 59afee7 to c65396e Compare June 20, 2025 12:55
@github-actions
Copy link
Copy Markdown

No changes detected in rendered charts.

@samsimpson1 samsimpson1 force-pushed the samsimpson1/chart-diff branch from c65396e to c12232a Compare June 20, 2025 12:58
@samsimpson1 samsimpson1 force-pushed the samsimpson1/chart-diff branch 10 times, most recently from dfa9e40 to f37f45b Compare June 20, 2025 14:01
@samsimpson1 samsimpson1 force-pushed the samsimpson1/chart-diff branch from f37f45b to a2e40c2 Compare June 20, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants